Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

do not run for PullRequest #318

Merged
merged 1 commit into from
Aug 31, 2014
Merged

do not run for PullRequest #318

merged 1 commit into from
Aug 31, 2014

Conversation

k-okada
Copy link
Contributor

@k-okada k-okada commented Aug 31, 2014

No description provided.

@k-okada k-okada mentioned this pull request Aug 31, 2014
Merged
@k-okada
Copy link
Contributor Author

k-okada commented Aug 31, 2014

Test PASSed.
Refer to this link for build results: http://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080job/hrpsys-qnx/1583/

@k-okada
Copy link
Contributor Author

k-okada commented Aug 31, 2014

アレ？これもダメですか？

@k-okada
Copy link
Contributor Author

k-okada commented Aug 31, 2014

間違っていました．直してPRを作り直しました．

@k-okada
Copy link
Contributor Author

k-okada commented Aug 31, 2014

Test PASSed.
Refer to this link for build results: http://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080job/hrpsys-qnx/1585/

@k-okada
Copy link
Contributor Author

k-okada commented Aug 31, 2014

Test PASSed.
Refer to this link for build results: http://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080job/hrpsys-qnx/1587/

@k-okada
Copy link
Contributor Author

k-okada commented Aug 31, 2014

Test PASSed.
Refer to this link for build results: http://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080job/hrpsys-qnx/1588/

@k-okada
Copy link
Contributor Author

k-okada commented Aug 31, 2014

Test PASSed.
Refer to this link for build results: http://jenkins.jsk.imi.i.u-tokyo.ac.jp:8080job/hrpsys-qnx/1589/

@k-okada
Copy link
Contributor Author

k-okada commented Aug 31, 2014

なおった気がします．テスト通ったらマージして
#317
等のテストも通るか確認してもらえればと思います．

fkanehiro added a commit that referenced this pull request Aug 31, 2014
@fkanehiro
Merge pull request #318 from k-okada/do_not_run_pr
e0b2f80 
do not run for PullRequest
@fkanehiro fkanehiro merged commit e0b2f80 into fkanehiro:master Aug 31, 2014
@fkanehiro
Copy link
Owner

相変わらず

fatal: empty ident name (for <>) not allowed
find: `/home/travis/.ros/test_results': No such file or directory
=== /home/travis/.ros/log/rostest-* ===
cat: $file: No such file or directory

というのは
https://travis-ci.org/fkanehiro/hrpsys-base/jobs/33977511
なんかででてますけど、これはいいんですかね？

@k-okada
Copy link
Contributor Author

k-okada commented Aug 31, 2014

@snozawa
#317 で上のPRを含んだ形で作りなおしてもらえますでしょうか？

git fetch --all
git checkout master
git pull origin master
git checkout revive_old_kf_codes
git merge  origin
git push snozawa revive_old_kf_codes

みたいにしているけど，もっと簡単な方法があれば教えて下さい．

snozawa added a commit to snozawa/hrpsys-base that referenced this pull request Sep 1, 2014
@snozawa
(.travis.yml) : Fix travis.yml reported in fkanehiro#318
06e1b48
@snozawa snozawa mentioned this pull request Sep 1, 2014
Merged
@k-okada k-okada deleted the do_not_run_pr branch February 17, 2015 05:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@k-okada @fkanehiro